Skip to content
This repository has been archived by the owner on Aug 11, 2021. It is now read-only.

Drop not needed dependency #41

Open
wants to merge 1 commit into
base: latest
Choose a base branch
from

Conversation

medikoo
Copy link

@medikoo medikoo commented Oct 17, 2019

Relying on external resolve-from seems unjustified, when in this given case same can be achieved via require.resolve.

Additional issue (which actually brought me here) is that resolve-from module references are not automatically traced by tools as pkg (while for require.resolve they are). That in turn requires additional manual setup when e.g. we want to bundle npm with our project into a single binary.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant